Skip to content

task(): remove 3rd-party auth deeplink functionality #19903

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
toufali merged 1 commit into from
Jan 21, 2026
Merged

task(): remove 3rd-party auth deeplink functionality #19903

toufali merged 1 commit into from
Jan 21, 2026
+38 −293

Conversation

@toufali
Copy link
Member

@toufali toufali commented Jan 16, 2026

Because

  • Adding the 3rd-party auth deeplink query param was only introduced for the Pocket user migration

This pull request

  • removes that functionality

Issue that this pull request solves

Closes: FXA-8873

@toufali toufali requested a review from a team as a code owner January 16, 2026 23:13
vbudhram
vbudhram approved these changes Jan 21, 2026
View reviewed changes
Copy link
Contributor

@vbudhram vbudhram left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@toufali Thank you!

@toufali toufali merged commit a5fb588 into main Jan 21, 2026
21 checks passed
@toufali toufali deleted the remove-deeplink branch January 21, 2026 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vbudhram vbudhram vbudhram approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@toufali @vbudhram